Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2048

multiselection: Fix the select_range implementation

When exclusive is TRUE, we would not always emit a
::selection-changed signal that covers all the items
that were unselected.

This commit includes a test.

set: Fix gtk_set_get_max

This function had an off-by-one error. Thankfully,
our selection tests are thorough enough to catch it.

dropdown: Connect a forgotten signal handler

The code was exporting a search_stop function, but
the ui file didn't connect it to the ::stop-search
signal. Fix that.

Merge branch 'frame-clock-start-time' into 'master'

Tweaks to frame clock for better ties to vsync

See merge request GNOME/gtk!2043

Merge branch 'master.msvc' into 'master'

Fix latest GTK master on Visual Studio

See merge request GNOME/gtk!2028

Merge branch 'columnview-reorder-cells' into 'master'

Columnview reorder cells

See merge request GNOME/gtk!2047

Merge branch 'rubberband-again' into 'master'

Rubberband again

See merge request GNOME/gtk!2041

multiselection: Fix the select_range implementation

When exclusive is TRUE, we would not always emit a
::selection-changed signal that covers all the items
that were unselected.

This commit includes a test.

set: Add a few useful functions

Allow to find out if a set is empty, and its
min and max.

columnview: Update column positions on reordering

Tell reordered columns to reorder their cells to
the new position. This is necessary to get things
like separators right. The visible symptom of this
problem was the lack of the right border when the
last column is reorder to another position, since
the title widget was still the last in its container,
so :last-child applied.

columnviewcolumn: Add private api to set the position

This is needed to make columns reorder their cells when
their position in the column views list of columns
changes.

listitemwidget: Add api to reorder children

We need this to keep the rows in sync with the columns
when they are reordered.

columnviewlayout: Handle for_size propertly

When we are given a for_size as width for the whole
column view, we need to distribute it over the columns
as gtk_column_view_allocate_columns would, in order
to find out which for_size to give to each cell.

This is a bit recursive, but works. Since we are
doing this recursion for every row, we should consider
adding a cache for those distributed widths.

Merge branch 'wip/damaged-but-no-frost-bite' into 'master'

x11: defer _NET_WM_FRAME_DRAWN update until frame usable by compositor

See merge request GNOME/gtk!1982

Merge branch 'matthiasc/for-master' into 'master'

gtk-demo: Bring back the applauncher demo

See merge request GNOME/gtk!2040

Merge branch 'ebassi/for-master' into 'master'

Add missing annotation

See merge request GNOME/gtk!2046

columnview: Split gtk_column_view_allocate_columns

Split off a helper function that just distributes
the widths, without actually allocating the columns.

This will be used in measuring in the future.

columnview: Take column width into account for measuring

When measuring the height of a cell, we need to give
it it the width of its column, if we expect realistic
results.

x11: Defer _NET_WM_FRAME_DRAWN update until frame usable by compositor

With the vendor provided Nvidia driver there is a small window of time
after drawing to a GL surface before the updates to that surface
can be used by the compositor.

Drawing is already coordinated with the compositor through the frame
synchronization protocol detailed here:

https://fishsoup.net/misc/wm-spec-synchronization.html

Unfortunately, at the moment, GdkX11Surface tells the compositor the
frame is ready immediately after drawing to the surface, not later,
when it's consumable by the compositor.

This commit defers announcing the frame as ready until it's consumable
by the compositor. It does this by listening for the X server to announce
damage events associated with the frame drawing.  It tries to find the
right damage event by waiting until fence placed at buffer swap time
signals.

x11: Factor out some of frame sync code into subroutines

This commit moves some of the end frame sync counter handling
code to subroutines.

It's a minor readability win, but the main motivation is to
make it easier in a subsequent commit to defer updating the
sync counter until a more appropriate time.

x11: Add back support for the damage extension

commit 14bf58ec5dfdf19e3ca603b977626608dafc729b dropped support
for using the DAMAGE extension since there was no code that
needed it.

We're going to need it again, however, to address an NVidia
vendor driver issue.

This commit does the plumbing to add it back.

filechooserwidget: Avoid an uninitialised value

For NULL filters, we were never calling g_list_store_find, so leaving
filter_index undefined.

demos: Avoid shadowing a global from time.h

Sad but true.

filterlistmodel: Fix fallthrough annotations in unreachable code

They are only unreachable in debug builds but still break the build on
clang.

columview: Make cells propagate request mode

I think this is necessary to make wrapping
labels work.

gtk-demo: Improve wrapping in settings demo

The columnview overrides the width of its content,
so if a label is not willing to wrap its content
below a certain width, it will just get cut off.

Avoid that by not setting width-chars on the wrapping
content.

Add missing annotation

columviewlayout: Skip children when we should

Every layout manager should call gtk_widget_should_layout
on their children before measuring or allocating them.
So do it here too.

listview: Add an example to the docs

The example is an excerpt from the applauncher
demo in gtk4-demo.

gtk-demo: Bring back the applauncher demo

Just without the coverflow. It was a well-documented
demo, so it is useful to keep around.

frame clock: schedule in refresh intervals

When the application does not receive "frame drawn" signals we schedule
the clock to run more or less at intervals equal to the last known
refresh interval. In order to minimize clock skew we have to aim for
exact intervals.

frame clock: Better handle non-regular clock cycles

We try to step the frame clock in whole refresh_interval steps, but to
avoid drift and rounding issues we additionally try to converge it to
be synced to the physical vblank (actually the time we get the
frame-drawn message from the compositor, but these are tied together).

However, the convergence to vsync only really makes sense if the new
frame_time actually is tied to the vsync. It may very well be that
some other kind of event (say a network or mouse event) triggered
the redraw, and not a vsync presentation.

We used to assume that all frames that are close in time (< 4 frames
apart) were regular and thus tied to the vsync, but there is really no
guarantee of that. Even non regular times could be rapid.

This commit changes the code to only do the convergence-to-real-time
if the cause of the clock cycle was a thaw (i.e. last frame drawn and
animating). Paint cycles for any other kind of reason are always
scheduled an integer number of frames after the last cycle that was
caused by a thaw.

frame clock: Track if paint is caused by thaw or not

When we get to a paint cycle we now know if this was caused by a
thaw, which typically means last frame was drawn, or some other event.

In the first case the time of the cycle is tied to the vblank in some
sense, and in the others it is essentially random. We can use this
information to compute better frame times. (Will be done in later
commits.)

Merge branch 'cleanup-snapshot-inhibit' into 'master'

reftest: Clean up reftest_inhibit_snapshot()

See merge request GNOME/gtk!2034

Drop gtk_multi_selection-copy

It is not used anymore.

listbase: Redo rubberbanding

Make it so that the selection is only updated in the end.

demos/gtk-demo/listview*.c: Include stdlib.h

atoi() and qsort() both require this header for them to be defined

modules: Fix build on Visual Studio

Visual Studio does not allow decorating functions with '__declspec (dllexport)'
if a prototype exists and is not decorated with '__declspec (dllexport)' as
well, so we cannot just decorate g_io_module_[load|unload|query] in the various
module sources with G_MODULE_EXPORT because the prototypes of these functions
have been marked with _GLIB_EXTERN, which equates to 'extern' unless overridden

Fix this by overriding _GLIB_EXTERN with the appropriate visibility flag, as we
have used to define _GDK_EXTERN.  Unfortunately, we can't just use _GDK_EXTERN
G_MODULE_EXPORT as they may have not been defined yet for our use

Do this across the board for all modules, even if they are not buildable on
Visual Studio nor Windows, for consistency's sake.

Merge branch 'fix-quadratic-add' into 'master'

Avoid quadratic slowdown in gtk_widget_add()

See merge request GNOME/gtk!2027

Add gtk_selection_model_[un]select_callback

Add a methods to add or remove a whole set
(specified via a query-range style callback).

set: Add a few useful functions

Allow to find out if a set is empty, and its
min and max.

Merge branch 'matthiasc/for-master' into 'master'

more documentation for list widgets

See merge request GNOME/gtk!2039

Merge branch 'fribidi-include' into 'master'

wayland: Remove unused fribidi include

See merge request GNOME/gtk!2037

docs: Expand list widget docs

Begin to flesh out the long descriptions for GtkListView,
GtkGridView and GtkColumnView.

wayland: Remove unused fribidi include

The included fribidi header is not used in gdkkeys-wayland.c and already
included in gdk.c which causes linker issues due to the header defining
a global variable.

gtk-demo: Cosmetics

Remove a few instances of double semicolon.

Merge branch 'matthiasc/for-master' into 'master'

gtk-demo: Allow editing in the settings demo

See merge request GNOME/gtk!2033

Merge branch 'list-widget-styles' into 'master'

List widget styles

Closes #2818

See merge request GNOME/gtk!2035

Merge branch 'demo-drop-awards' into 'master'

gtk-demo: Drop the awards

See merge request GNOME/gtk!2036

Avoid quadratic slowdown in gtk_widget_add()

If you add a widget to a parent, this will invalidate the css nodes
for parent/siblings. Afterwards, if the parent is mapped, we will
realize the new child. This calls gtk_widget_update_alpha() which
needs the css opacity, so it revalidates the css.

Thus, for each widget_add (while visible) will trigger a full
revalidation of each sibling. If you add N children to a parent that
leads to O(N^2) revalidations.

To demo this I changed gtk-demo to always double the count
(independent of the fps) and print the time it took. Here is the
results (after a bit):

Setting fishbowl count=256 took 3,4 msec
Setting fishbowl count=512 took 10,1 msec
Setting fishbowl count=1024 took 34,1 msec
Setting fishbowl count=2048 took 126,3 msec
Setting fishbowl count=4096 took 480,3 msec
Setting fishbowl count=8192 took 1892,7 msec
Setting fishbowl count=16384 took 7751,0 msec
Setting fishbowl count=32768 took 38097,7 msec
Setting fishbowl count=65536 took 191987,7 msec

To fix this we drop gtk_widget_update_alpha() and just
calculate it when needed (which is only in a single place).
It was really only necessary because we previously set
the alpha on the surface.

With this fix the above becomes:

Setting fishbowl count=256 took 1,0 msec
Setting fishbowl count=512 took 1,9 msec
Setting fishbowl count=1024 took 3,7 msec
Setting fishbowl count=2048 took 7,4 msec
Setting fishbowl count=4096 took 18,1 msec
Setting fishbowl count=8192 took 31,0 msec
Setting fishbowl count=16384 took 66,3 msec
Setting fishbowl count=32768 took 126,7 msec
Setting fishbowl count=65536 took 244,6 msec
Setting fishbowl count=131072 took 492,2 msec
Setting fishbowl count=262144 took 984,3 msec

gtk-demo: Drop the awards

This was a neat idea, but maintaining it at a sufficient
level is too much work.

list widgets: Document css structure

We didn't fill in this expected part of the widget
documentation yet.

Merge branch 'ensure-style-no-recurse' into 'master'

Avoid recursion in gtk_css_node_ensure_style()

See merge request GNOME/gtk!2031

docs: Fix a parameter name mismatch

Change css names of list widget

The new names are

GtkListView - listview row
GtkGridView - gridview child
GtkColumView - columnview header
               columnview listview row

Adwaita css has been updated to preserve
existing styles.

Fixes: #2818

reftest: Clean up reftest_inhibit_snapshot()

This was done in a weird way where we always call reftest_uninhibit_snapshot()
on paint, and then re-inhibited it if it wasn't inhibited. To make this
work it also started with an extra inhibit.

This is very contorted and based on how this historically worked. This
changes it to just do:

  if (inhibit_count > 0)
    return;

And keep inhibit_count at its initial zero value unless it is actually
inhibited.

Merge branch 'fix-snapshot' into 'master'

snapshot: Fix assert if paint gets scheduled immediately

See merge request GNOME/gtk!2032

gtk-demo: Allow editing in the settings demo

This gives us an example of a columnview with editable
content, which we didn't have so far.

snapshot: Fix assert in inhibition

In https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/2027 i was getting

Bail out! ERROR:../testsuite/reftests/reftest-snapshot.c:212:reftest_uninhibit_snapshot: assertion failed: (inhibit_count > 0)

In (for example the box-shadow-changes-modify-clip reftest. I can reproduce this (on master) with:

```
$ xvfb-run -a -s "-screen 0 1024x768x24" meson test --suite gtk:reftest "reftest box-shadow-changes-modify-clip.ui"
...
1/1 gtk:reftest / reftest box-shadow-changes-modify-clip.ui ERROR          0.77s
``

Fix this by re-inhibiting if we didn't draw anything, or we will get an assert the next paint.

Merge branch 'kjellahl/flowboxremove' into 'master'

flowbox: Don't use a removed and destroyed child

See merge request GNOME/gtk!2029

Merge branch 'fishbowl-children' into 'master'

gtk-demo: Avoid list for children in GtkFishBowl

See merge request GNOME/gtk!2030

Avoid recursion in gtk_css_node_ensure_style()

gtk_css_node_ensure_style() recurses over previous siblings to ensure
these have a style before its following sibling.  As seen in
https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/2027 this can
cause us to stack overflow and crash if we have a lot of children.

And even if we don't have *that* many children its still somewhat
bad to have stack depths of the same magnitude as the number of
children, both for performance reasons and debuggability.

gtk-demo: Avoid list for children in GtkFishBowl

This is a demo that measures performance, so keep the child
info in a hashtable instead of a list. This means adding or removing
a child is not O(n-children).

flowbox: Don't use a removed child after it has been unparented

In gtk_flow_box_remove(), call g_sequence_remove() before the child is unparented.

See MR !2029

Merge branch 'santo/gtk4_spelling_fix' into 'master'

docs: Fix spelling of GDK_VERSION_MIN_REQUIRED.

See merge request GNOME/gtk!2026

Updated Spanish translation

Updated Spanish translation

docs: Fix spelling of GDK_VERSION_MIN_REQUIRED.

Merge branch 'columnview-separators' into 'master'

Columnview separators

See merge request GNOME/gtk!2025

columnview: Add a property for column separators

Rename the show-separators property to show-row-separators,
and add a matching show-column-separators property. It is
implemented by setting the .column-separators style class
on the column view.

listview: Use gtk_widget_add_css_class

No need to use the style context api for this anymore.

settings demo: Don't select rows

We want no selection here, so use GtkNoSelection.

Merge branch 'columnview-printdialog' into 'master'

Port the printdialog to column view

See merge request GNOME/gtk!2010

Merge branch 'columnview-rubberbanding' into 'master'

columnview: Implement rubberbanding

See merge request GNOME/gtk!2008

Merge branch 'columnview-expand' into 'master'

columnview: Implement expanding columns

See merge request GNOME/gtk!2007

Merge branch 'columnview-reordering' into 'master'

columnview: Implement interactive reordering

See merge request GNOME/gtk!2006

Merge branch 'columnview-resizing' into 'master'

columnview: Implement interactive resizing

See merge request GNOME/gtk!2004

Merge branch 'columnview-scrolling' into 'master'

columnview: Implement horizontal scrolling

See merge request GNOME/gtk!1998

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!2024

printdialog: Port to column view

gtk-demo: Make gridview demo use rubberbanding

Add rubberband api

Add an ::enable-rubberband property to GtkListView,
GtkGridView and GtkColumnView.

listbase: Simplify rubberbanding

The new approach is:
 - plain: clear and start a new selection
 - extend: add to the existing selection
 - modify: subtract from the existing selection

listbase: Split off rubberband data

listbase: Use a widget for the rubberband

We no longer need to juggle manual css nodes. Just create
a widget for the rubberband, and update its allocation
as we rubberband.

Add autoscroll

This is an expected feature with rubberband selection:
as you get close to the edge while doing rubberband
selection, the list scrolls to extend your selection.

listbase: Add rubberband selection

Implement the typical rubberband selection, including
autoscroll. This is only useful with multiselection,
and not very compatible with single-click-activate.
Therefore, it is not enabled by default, and needs
to be turned on explicitly.

Add GtkMultiSelection

This is implemented using a private GtkSet helper.

Includes tests.

gtk-demo: Expand columns in the settings demo

Expand the columns that are likely to appear
at the end.

inspector: Expand the actions list

inspector: Expand the resource list

This is an experiment with adding a filler column.

inspector: Expand the property list

It looks better this way.

inspector: Expand the object tree

This is how it used to look, and it looks better that way.

columnview: Take expand into account

When allocating columns, distribute extra space
to columns that have expand set to TRUE.

columnview: Add a GtkColumnViewColumn:expand property

This will be used to determine how to distribute
available extra space in a column view.

columnview: Add autoscroll

Autoscroll when the pointer gets close to the
edge during column resizing or reordering. This
is similar to what the treeview does, but it is
implemented using a tick callback, and has
variable speed.

columnview: Allow to cancel reorder with Escape

The treeview does this too.

columnview: Interactive column reordering

Allow rearranging columns by dragging, in the same
way the treeview does.

We add the "dnd" style class to the header while
it is dragged, and we move the header of the dragged
column to the end of its parents children, so that
it gets drawn on top.

columnview: Add a GtkColumnView:reorderable property

This property controls if users can reorder columns
by drag-and-drop. It defaults to TRUE.